-
Notifications
You must be signed in to change notification settings - Fork 177
Added complete e2e tests automation framework #3634
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
📦 Latest Plugin BuildBuilt at: 2025-09-24T23:17:46.426Z Download: Click here to download the plugin To download: Click the link above → Scroll to bottom → Download "facebook-for-woocommerce" artifact |
| wp option update wc_facebook_external_business_id "${{ secrets.FB_EXTERNAL_BUSINESS_ID }}" --allow-root | ||
| wp option update wc_facebook_product_catalog_id "${{ secrets.FB_PRODUCT_CATALOG_ID }}" --allow-root | ||
| wp option update wc_facebook_pixel_id "${{ secrets.FB_PIXEL_ID }}" --allow-root | ||
| wp option update wc_facebook_has_connected_fbe_2 "yes" --allow-root |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The following rows we might also wanna add as repo variables
|
@immadhavv has imported this pull request. If you are a Meta employee, you can view this in D83242501. |
|
@immadhavv has imported this pull request. If you are a Meta employee, you can view this in D83242501. |
|
@immadhavv merged this pull request in 8efa689. |
Summary: * Tweak - Focus admin user check on order creation capabilities by iodic in #3643 * Fix - Added code to exclude purchases registered by the admin user by vahidkay-meta in #3641 * Add - [WIFI][EZ] Localization Fix on WhatsApp Splash page by sharunaanandraj in #3642 * Tweak - Removed get_product_fbid graph api functionality by vinkmeta in #3624 * Dev - Plugin availability logging by rafael-curran in #3637 * Add - Added complete e2e tests automation framework by immadhavv in #3634 * Fix - Fix/issue with set stable tag by vahidkay-meta in #3636 * Fix - Removed the new changes by vahidkay-meta in #3635 * Tweak - Removed API call to delete products by vinkmeta in #3623 * Tweak - Cleanup deprecated product delete functionality by vinkmeta in #3622 * Fix - [EZ] Error Message for Customer Events API call by woo-ardsouza in #3630 Pull Request resolved: #3648 Reviewed By: ajello-meta, devbodaghe Differential Revision: D83758828 Pulled By: rafael-curran fbshipit-source-id: 3ab59df5ae1ab0e1888b097f67e98fc94269e828
Description
Existing E2E tests were only being run and tested on the client side , but not tested if they got synced to Meta properly. This was done by the QA team manually as we did not have any proper framework to do this automatically.
This change enables the team to do just that, by introducing a framework and two critical e2e tests (simple product creation, variable product creation) using the framework to test everything end to end. More tests that are run manually currently can be automated using the same framework.
Changes -
Updated the plugin installation to use the required config to connect to a test Meta Catalog. This enables the sync.
Fixed the failing e2e tests and updated the tests (to use github_run_id to uniquely identify products from a run and avoid collision) to verify the sync entirely using a Validator
Added a Validator that just takes a productId and verifies it completely e2e
Finally after validation, clean up the product from Meta Catalog.
Type of change
Checklist
Changelog entry
added QA automation framework to completely automate e2e tests.
Test Plan
https://github.com/facebook/facebook-for-woocommerce/actions/runs/17991535339
Screenshots
Before
NA
After
e2e-tests-validated2.mov